Skip to content

fix unusable scroll bar on user list#619

Open
ZmagoD wants to merge 1 commit intostride-nyc:masterfrom
ZmagoD:fix-unusable-scroll-bar-on-user-list-overflow
Open

fix unusable scroll bar on user list#619
ZmagoD wants to merge 1 commit intostride-nyc:masterfrom
ZmagoD:fix-unusable-scroll-bar-on-user-list-overflow

Conversation

@ZmagoD
Copy link
Copy Markdown

@ZmagoD ZmagoD commented Oct 28, 2023

Description of Change(s) Introduced:
Fixes the overflow issue on the users list. Related to bug report #617

Relevant Screenshots/GIFs:
Screenshot 2023-10-28 at 22 10 28
Screenshot 2023-10-28 at 22 11 01

Relevant github Issue:

@CLAassistant
Copy link
Copy Markdown

CLAassistant commented Oct 28, 2023

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

vanderhoop
vanderhoop reviewed Jun 14, 2024
View reviewed changes
web/static/js/components/css_modules/user_list.css
min-height: 8rem;
display: inline-block;
white-space: initial;
overflow-x: auto;
Copy link
Copy Markdown
Contributor

@vanderhoop vanderhoop Jun 14, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hmmm... the user list is already overflow-x: auto;, so it is scrollable, but I can see how that isn't apparent on the UI, depending on the user's OS settings for showing scroll gutters.

Does this PR cause largeer user lists to expand on other board layouts like in Idea Submission? The photo in the PR summary only shows the prime directive layout

@JohnJasonFisher
Copy link
Copy Markdown
Contributor

JohnJasonFisher commented Mar 17, 2026

IMO, this is an improvement on the prime directive layout page and it doesn't effect other pages like Idea Submission.

JohnJasonFisher added a commit that referenced this pull request Mar 18, 2026
@JohnJasonFisher
Add css fix from user submitted issue
5610a38 
#619
JohnJasonFisher added a commit that referenced this pull request Mar 19, 2026
@JohnJasonFisher
Add css fix from user submitted issue
32d5b9e 
#619
JohnJasonFisher added a commit that referenced this pull request Mar 20, 2026
@JohnJasonFisher
Add css fix from user submitted issue
6da2b7c 
#619
JohnJasonFisher added a commit that referenced this pull request Mar 20, 2026
@JohnJasonFisher
Add css fix from user submitted issue
adf70b5 
#619
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@vanderhoop vanderhoop vanderhoop left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@ZmagoD @CLAassistant @JohnJasonFisher @vanderhoop